Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

장부, 봉투 생성 시 상세 화면 이동 / 받은 장부 목록 쿼리 수정 #168

Merged
merged 4 commits into from
May 15, 2024

Conversation

jinukeu
Copy link
Member

@jinukeu jinukeu commented May 15, 2024

💡 Issue

  • Resolved: #이슈번호

🌱 Key changes

장부, 봉투 생성 시 상세 화면 이동 / 받은 장부 목록 쿼리 수정

✅ To Reviewers

📸 스크린샷

스크린샷
파일첨부바람

@jinukeu jinukeu added the QA label May 15, 2024
@jinukeu jinukeu requested a review from yangsooplus May 15, 2024 06:21
@jinukeu jinukeu self-assigned this May 15, 2024
@@ -20,7 +20,7 @@ interface LedgerService {
@Query("title") title: String?,
@Query("categoryIds") categoryIdList: List<Int>?,
@Query("fromStartAt") fromStartAt: LocalDateTime,
@Query("toEndAt") toEndAt: LocalDateTime,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

스웨거 보니까 toEndAt도 남아있던데, toStartAt이 toEndAt를 완전히 대체하는 것인가요? 아님 toStartAt이랑 toEndAt이랑 다른 쿼리인가요?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

완전히 바뀔 예정인걸로 알아요!

자세한 히스토리는 ... https://www.notion.so/489e952079c9405f9322a18eabe01084?pvs=4

요거 참고 부탁드림다

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아아 이거 관련이엇군요 친절한 링크제공 감사합니다

@jinukeu jinukeu force-pushed the feature/move-create-envelope-ledger branch from 0c600db to 1db85b0 Compare May 15, 2024 10:10
@jinukeu jinukeu merged commit 3a44819 into develop May 15, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants